Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HEENDY-47-review-image] 리뷰 작성 시, 다중 이미지 업로드(S3) 구현 #28

Merged
merged 1 commit into from
Mar 3, 2024

Conversation

sooyoungh
Copy link
Member

@sooyoungh sooyoungh commented Mar 3, 2024

구현 사항

  • 리뷰 작성 시, 다중 이미지 업로드 기능 구현 (리뷰 내용은 필수, 이미지는 선택!)
  • POSTMAN 데탑 버전만 파일 업로드되고 브라우저는 로컬 파일이 안됨 참고해주세욥
image
  • ‼️ 리뷰에 이미지 추가 시, 리뷰 저장 전 이므로 아직 reviewId(PK)를 몰라서 이미지 테이블에 reviewId 추가 못함
    => 리뷰 저장하고, 이미지는 그 이후에 저장할까 했지만, 한번에 트랜잭션 처리하는 게 우선인 것 같아서
    => 일단 FK 설정 제외

image

참고 사항

@sooyoungh sooyoungh self-assigned this Mar 3, 2024
@sooyoungh sooyoungh added 💡 feature 기능 개발 👻 팻치 @sooyoungh labels Mar 3, 2024
@sooyoungh sooyoungh changed the title feat: 리뷰 작성 시, 다중 이미지 업로드(S3) 구현 [HEENDY-47-review-image] 리뷰 작성 시, 다중 이미지 업로드(S3) 구현 Mar 3, 2024
Comment on lines +35 to +37

private final AwsS3Config awsS3Config; // TODO: AmazonS3 싱글톤으로 정의하기

Copy link
Member Author

@sooyoungh sooyoungh Mar 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sangeun99 AmazonS3 싱글톤으로 정의할까요?
+) S3 config에 정의해주고, 기능 메소드들은 서비스단으로 분리해도 좋을것 같습니다 ㅎㅎ

@sooyoungh sooyoungh added the 🔥 H-5 5시간 안에 리뷰해주세요 label Mar 3, 2024
@sooyoungh sooyoungh merged commit a5257b6 into dev Mar 3, 2024
2 checks passed
@sooyoungh sooyoungh deleted the HEENDY-47-review-image branch March 3, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👻 팻치 @sooyoungh 💡 feature 기능 개발 🔥 H-5 5시간 안에 리뷰해주세요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant